Skip to content

ci: remove ref option from prettier workflow checkout #733

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Achanandhi-M merged 5 commits into from
Dec 15, 2025
Merged

ci: remove ref option from prettier workflow checkout #733

Achanandhi-M merged 5 commits into from
Dec 15, 2025

Conversation

@manasmanohar
Copy link
Contributor

@manasmanohar manasmanohar commented Dec 15, 2025 •
edited
Loading

What has changed?

  • Removed the ref option from the checkout step.
  • check code formatting only for changed files in pull requests.
  • remove auto commit prettier change

This PR Resolves #(3375)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue).
  • New feature (non-breaking change which adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to not work as expected).
  • Documentation update (if none of the other choices apply).

How Has This Been Tested?

Please run npm run build and npm run serve to check if the changes are working as expected. Please include screenshots of the output of both the commands. Add screenshots/gif of the changes if possible.

Checklist:

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.

@manasmanohar
Remove ref option from checkout step
c37eea7 
Removed the ref option from the checkout step.

Signed-off-by: Manas Manohar <[email protected]>
@manasmanohar manasmanohar self-assigned this Dec 15, 2025
@manasmanohar manasmanohar added the bug Something isn't working label Dec 15, 2025
@manasmanohar
Enhance Prettify Code workflow for changed files
961ea94 
Updated the GitHub Actions workflow to fetch the base branch and check code formatting only for changed files in pull requests.

Signed-off-by: Manas Manohar <[email protected]>
@manasmanohar
test: prettier workflow
05d69ae 
Signed-off-by: Manas Manohar <[email protected]>
@manasmanohar
chore: use variable for changed files
44e0437 
Signed-off-by: Manas Manohar <[email protected]>
@manasmanohar
undo prettier testing changes
c9447c3 
Removed unnecessary blank lines and ensured proper formatting.

Signed-off-by: Manas Manohar <[email protected]>
Achanandhi-M
Achanandhi-M approved these changes Dec 15, 2025
View reviewed changes
Copy link
Member

@Achanandhi-M Achanandhi-M left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Achanandhi-M Achanandhi-M merged commit 57ad5e8 into main Dec 15, 2025
4 of 7 checks passed
@Achanandhi-M Achanandhi-M deleted the fix/prettier-workflow-branch-ref branch December 15, 2025 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Achanandhi-M Achanandhi-M Achanandhi-M approved these changes

Assignees

@manasmanohar manasmanohar

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@manasmanohar @Achanandhi-M